Skip to content

fix(test): ограничить Angular deps.inline средой Windows - #389

Merged
TThanos3000 merged 1 commit into
devfrom
fix/dev-vitest-windows-inline
Sep 28, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
fix/dev-vitest-windows-inline

Conversation

@TThanos3000

Copy link
Copy Markdown
Contributor

Что изменено

В vitest.config.ts workaround test.server.deps.inline применяется только при process.platform === "win32". Linux использует обычную конфигурацию без этого блока.

Зачем

Глобальный inline после #387 вызывал 6 OOM в Linux vmThreads. При его удалении тот же feature code и все новые тесты прошли: 400/400 files, 1934 tests, 0 unhandled errors. На Windows без workaround воспроизводится NG0401 в общем TestBed setup на Node 20/24.

Реализация

Чистая ветка от origin/dev (cda836a2), один commit 27901cd9, один изменённый файл. Содержимое vitest.config.ts идентично проверенному a4abf18c. Штатный dev workflow сохранён с npm run test:ci; временные feature triggers, forks/maxWorkers и deploy guard из диагностической ветки не переносились.

API

Не менялся.

UI

Runtime и case feature не менялись.

Permissions

Не менялись.

Проверки

  • Diff с dev: только vitest.config.ts; git diff --check PASS. Tests, exclusions, coverage, dependencies и heap settings не изменены.
  • Исходный Windows-only config: Windows Node 20 — 400/400 files, 1934 tests PASS, 0 unhandled/OOM.
  • Свежий GitHub CI с checkout точного commit 27901cd96f50d8efe73a5b68fd17e100162f5d48: 400/400 files, 1934 tests PASS, 0 unhandled, 0 OOM, 114.02 s. SHA подтверждён checkout log.
  • Ubuntu / Node 20: штатные npm run format:check, npm run lint:ts, npm run test:ci, npm run build:pr — PASS. Deploy skipped.

Acceptance criteria

  • Windows сохраняет workaround; Linux не получает новый inline.
  • Workflow dev не изменён; тестовая команда остаётся штатной.
  • Runtime/tests/coverage/exclusions и ngx-autosize не затронуты.
  • Полный CI: 400/400 files, 1934 tests PASS, 0 unhandled/OOM, build PASS.

Не проверено

В репозитории нет CI trigger для PR. Указанный успешный run создан в существующей диагностической ветке, но actions/checkout проверенно загрузил точный SHA этого PR и выполнил штатные npm-команды. Это не автоматический PR check; временная конфигурация запуска не входит в PR. macOS отдельно не проверялся; browser QA неприменим к этому test-config fix.

Риски / migration

Migration не требуется. Глобальная область workaround сужена до подтверждённой Windows-среды. Merge и deploy выполняет пользователь; диагностический запуск пропускает deploy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants